-
Notifications
You must be signed in to change notification settings - Fork 197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: constraints added to linting #3533
Conversation
|
🚀 Deployed on https://pr-3533--spectrum-css.netlify.app |
File metricsSummaryTotal size: 3.22 MB* 🎉 No changes detected in any packages * Size determined by adding together the size of the main file for all packages in the library.* Results are not gzipped or minified. * An ASCII character in UTF-8 is 8 bits or 1 byte. |
5dee0cf
to
13aae1d
Compare
13aae1d
to
603c775
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran through the validation steps and everything looks good to me!
Description
Remove the
bumpVersionsWithWorkspaceProtocolOnly
setting from the Changeset config and downgrade theupdateInternalDependencies
fromminor
topatch
to ensure local dependencies stay tightly aligned.Restructure the constraints file a bit to make it more readable and ensure it is fully documented.
Apply the constraints via
lint-staged
whenever a package.json file is updated.How and where has this been tested?
Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.
Validation steps
yarn constraints
: expect no changes to this because no functionality has changed. Should not get any console feedback because there are no suggested changes to the project. (@cdransf)Note: We'll have to validate the changeset update during the next release. We'll expect the next release to bump local devDependencies for all packages.
To-do list